Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Temporarily use ubuntu-22.04 instead of latest in CI #45

Merged
merged 1 commit into from
Dec 18, 2024
Merged

Temporarily use ubuntu-22.04 instead of latest in CI #45

merged 1 commit into from
Dec 18, 2024

Conversation

james-d-mitchell
Copy link
Member

This is due to ubuntu-latest being updated from ubuntu-22.04 to ubuntu-24.04 and there being different versions of gcc/clang available in ubuntu-24.04 and ubuntu-22.04. See the following for some details.

actions/runner-images#10636

A proper fix to this would be one which installs the relevant versions of gcc/clang in which ever version of ubuntu the runner is using. It's not completely clear to me how to do that at the moment.

@james-d-mitchell
Temporarily use ubuntu-22.04 instead of latest in CI
38f4e23 
This is due to ubuntu-latest being updated from ubuntu-22.04 to
ubuntu-24.04 and there being different versions of gcc/clang available
in ubuntu-24.04 and ubuntu-22.04. See the following for some details.

actions/runner-images#10636

A proper fix to this would be one which installs the relevant versions
of gcc/clang in which ever version of ubuntu the runner is using. It's
not completely clear to me how to do that at the moment.
@james-d-mitchell james-d-mitchell added the ci Label for ci related issues/PRs label Dec 18, 2024
@james-d-mitchell james-d-mitchell merged commit 8c587c1 into libsemigroups:main Dec 18, 2024
14 checks passed
@james-d-mitchell james-d-mitchell mentioned this pull request Dec 18, 2024
Merged
@Joseph-Edwards
Copy link

Including the following enables versions 9 to 14 of g++ to be installed onto Ubuntu 24.04:

sudo add-apt-repository ppa:ubuntu-toolchain-r/ppa -y

To the best of my knowledge, there is no way of installing versions 11 to 13 of clang++ onto Ubuntu 24.04 out of the box. As I see it, the possible options of to resolve this are:

  1. Build clang in the CI from the sources. Probably very slow, but may be able to be sped up with cacheing?
  2. Build the relevant versions of clang/llvm ourselves, host those pre-built binaries, and install these when running the tests in the CI.
  3. Install a pre-built binary from the source for an old version of Ubuntu, and hope everything still works. Probably a bad idea.
  4. Have some tests on the CI run on 22.04, and some run on ubuntu-latest.
  5. Scrap versions 13 and below on the CI.

I'd be happy to try and make a PR that implements any of the above.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
ci Label for ci related issues/PRs
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@james-d-mitchell @Joseph-Edwards